🤖 fix: track estimated costs for ChatGPT OAuth usage - #3914
Conversation
|
@codex review Please review the ChatGPT OAuth cost tracking fix. |
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 8b221518e8
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex review The security review reported no findings. I minimized that informational comment. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 8b221518e8
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
This comment has been minimized.
This comment has been minimized.
8b22151 to
d855b12
Compare
|
@codex review The PR now uses a forward-only scope. It tracks estimated costs for new ChatGPT OAuth requests. It does not migrate old ledgers or analytics rows. |
|
Codex Review: Didn't find any major issues. You're on a roll. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Summary
Track estimated model costs for new ChatGPT OAuth usage.
Background
ChatGPT OAuth usage records token counts but previously displayed zero cost.
The original behavior came from commit
9e60eb2cband PR#2171.That path marked OAuth usage as
costsIncluded.Implementation
costsIncludedbehavior for other provider routes.This PR does not migrate old chat rows, session ledgers, or analytics rows.
The displayed amount represents an estimated model-equivalent cost.
It does not represent a ChatGPT subscription invoice.
Validation
make static-checkRisks
$0.00.costsIncludedremain unchanged.Generated with
mux• Model:openai:gpt-5.6-luna• Thinking:xhigh• Cost:$0.04